Skip to content

Give the unexpected error toast its own title - #1392

Merged
microbit-matt-hillsdon merged 2 commits into
microbit-foundation:mainfrom
helloigig:unexpected-error-title
Nov 4, 2026
Merged

microbit-matt-hillsdon merged 2 commits into
microbit-foundation:mainfrom
helloigig:unexpected-error-title

Conversation

@helloigig

Copy link
Copy Markdown
Contributor

Closes #1267

The unexpected error toast used unexpected-error-description as its title. That message has a <link> tag and no link was passed for the title, so formatting failed and the toast (and screen readers) showed the raw id "unexpected-error-description" as the title.

  • The toast now uses unexpected-error-title, which reads "An unexpected error occurred", matching the full-page error view.
  • The existing translations of unexpected-error-title were copies of the description, <link> and all, so i18n:tidy --check fails once the English changes. I've removed just those entries so English shows until Crowdin has new ones. When uploading, please leave "keep translations" unticked for this string.
  • Added a small test for the toast title.

Checked with typecheck, lint, unit tests, i18n:tidy --check, the save, multiple-files and settings e2e tests, and a build. I also triggered the toast in the browser in English and German to confirm the title.

AI-assisted.

helloigig and others added 2 commits November 4, 2026 00:45
The toast used the description message, which has a <link> tag, as its
title without passing a link, so formatting failed and the title showed
the raw message id. Use unexpected-error-title instead and give it real
title text.

The existing translations of unexpected-error-title are copies of the
description, so they are removed and English shows until Crowdin has
new ones.

Closes microbit-foundation#1267

@microbit-matt-hillsdon microbit-matt-hillsdon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@microbit-matt-hillsdon
microbit-matt-hillsdon merged commit f88b88d into microbit-foundation:main Nov 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix unexpected-error-title text

2 participants